Add MCP adapters to autogen-ext #5251
Conversation
richard-gyiko
changed the title
|
@microsoft-github-policy-service agree |
|
Hi @richard-gyiko , Thanks for this PR .. |
|
Hi @victordibia , I've updated the sample on the community repo which now includes the fetch and the file system MCP server tools: MCP Server installations and usage is covered in the README and the sample code. Hope it gives sufficient amount of information. If not, please let me know. |
python/packages/autogen-ext/src/autogen_ext/tools/mcp/__init__.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Would this functionality be better served by something like the Workbench design rather than a one-off factory
There was a problem hiding this comment.
Workbench seems to be a higher-level concept to me. You could add the tools gathered with the factory method to the workbench if you wanted to. Am I missing something?
|
|
||
|
|
||
| @asynccontextmanager | ||
| async def create_mcp_server_session( |
There was a problem hiding this comment.
When running the mcp code locally I was running into the following bug: modelcontextprotocol/python-sdk#169. Have you run into it by any chance?
There was a problem hiding this comment.
I just ran this sample:
https://github.com/richard-gyiko/autogen-ext-mcp/blob/main/samples/agent_chat.py and I saw no sign of this bug.
Why are these changes needed?
The increasing popularity of MCP decided to migrate the community extension into the autogen-ext package.
Related issue number
Checks